Skip to content

feat: containerize frontend and harden CI for Module 14 delivery prep - #14

Merged
MohanEhab merged 3 commits into
mainfrom
devops/module-14-prep
Jul 17, 2026
Merged

feat: containerize frontend and harden CI for Module 14 delivery prep#14
MohanEhab merged 3 commits into
mainfrom
devops/module-14-prep

Conversation

@MohanEhab

Copy link
Copy Markdown
Contributor

Summary

  • Preparation work for Module 14 (deployment/CI foundation): containerizes the frontend and hardens CI, ahead of Module 14's own scoped work.

What changed

  • New Dockerfile, .dockerignore; next.config.ts adds output: "standalone" and a same-origin /api/:path* rewrite via SIMPLE_API_PROXY_TARGET.
  • CI hardening: SBOM generation via CycloneDX, pinned GitHub Actions to commit SHAs, container build + smoke test, npm audit --audit-level=high gate.
  • New .github/dependabot.yml, .github/workflows/codeql.yml.
  • README.md: new "Container and same-origin delivery" section; .env.local.example updated for the proxy target.

Tests

  • CI workflow itself is the validation surface (build, container build, smoke test, audit); no application test changes.

Security/docs

  • CodeQL and Dependabot added for ongoing scanning; SBOM published per CI run.

Risks/deferred

  • This is prep work, not full Module 14 delivery: Azure/staging deployment, gateway wiring, and release evidence remain scoped to Module 14 itself.
  • Branch history includes the pre-existing fix(module-06-lobby-matchmaking): resolve Active Lobby link from real membership commit, not yet separately PR'd against origin/main (also present on the module-07 PR branch).

… membership

The sidebar's Active Lobby link pointed at a hardcoded lobby code (SP-7F-29).
Resolve the user's active lobby from lobbyApi.getMyActive() and hide the link
when they are not in one. Also clear the stored lobby credential on leave and
add the test:coverage script.
Adds a production Dockerfile (standalone Next.js output, same-origin /api
proxy for Caddy), CodeQL/Dependabot scanning, and a hardened CI pipeline
(pinned actions, SBOM generation, container build/smoke test, npm audit
gate). Ahead of Module 14; not required by Module 7.
Non-breaking transitive bumps; CI's audit-level=high gate was failing on
undici (TLS/SSRF/DoS advisories) and vite (Windows path bypass advisory).
@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@MohanEhab
MohanEhab merged commit 1560519 into main Jul 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants